Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DCJ-621][risk=no] Small fixes for documentation issues found during onboarding. #2652

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

otchet-broad
Copy link
Contributor

@otchet-broad otchet-broad commented Aug 19, 2024

Addresses

https://broadworkbench.atlassian.net/browse/DCJ-621

Summary

Two small items needed to be addressed while onboarding: node version instructions and a comment in CSS.

  1. DEVNOTES.md file now reflects the usage of node 22.6.0
  2. bootstrap_replacement.css contained a comment with a# character. This leads to an error in the source map loader. I've updated the comment and removed the offending character so that the comment remains and the source map loader works properly and stops complaining about it.

Have you read Terra's Contributing Guide lately? If not, do that first.

  • Label PR with a Jira ticket number and include a link to the ticket
  • Label PR with a security risk modifier [no, low, medium, high]
  • PR describes scope of changes
  • Get a minimum of one thumbs worth of review, preferably two if enough team members are available
  • Get PO sign-off for all non-trivial UI or workflow changes
  • Verify all tests go green
  • Test this change deployed correctly and works on dev environment after deployment

….0), and adjusts the comment in the css to clear the error in building the app.
@otchet-broad otchet-broad marked this pull request as ready for review August 19, 2024 17:55
@otchet-broad otchet-broad requested a review from a team as a code owner August 19, 2024 17:55
@otchet-broad otchet-broad requested review from rushtong and fboulnois and removed request for a team August 19, 2024 17:55
Copy link
Contributor

@rushtong rushtong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 👍🏽


```
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash
nvm install 16
nvm install 22.6.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd love it if we could tie this to what's defined in Dockerfile, but that's definitely out of scope here.

Copy link
Contributor

@fboulnois fboulnois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@otchet-broad otchet-broad merged commit 37f1265 into develop Aug 20, 2024
9 checks passed
@otchet-broad otchet-broad deleted the otchet/DCJ-621_small_fixes_to_documentation branch August 20, 2024 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants